Fix Persuasion Rack Bugs #1591
Merged
Fix Persuasion Rack Bugs #1591
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Prevents basicmob torture, while also preventing the user from spamming the rack (which would make them lose a lot of blood)
|
By the way, regarding the basic mob torture, that was another fix. For some reason, torturing them would instantly skip the bar and just instantly vassalize them. It seems like they were forgotten, leading me to believe they probably were not meant to be tortured. |
wraith-54321
reviewed
Apr 7, 2024
monkestation/code/modules/bloodsuckers/structures/bloodsucker_crypt.dm
Outdated
Show resolved
Hide resolved
monkestation/code/modules/bloodsuckers/structures/bloodsucker_crypt.dm
Outdated
Show resolved
Hide resolved
…crypt.dm Co-authored-by: wraith-54321 <[email protected]>
dwasint
reviewed
Apr 7, 2024
monkestation/code/modules/bloodsuckers/structures/bloodsucker_crypt.dm
Outdated
Show resolved
Hide resolved
monkestation/code/modules/bloodsuckers/structures/bloodsucker_crypt.dm
Outdated
Show resolved
Hide resolved
…t words. suggested by maintainer, changed it because I see no reason not to. plus looks nicer!!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Prevents basicmob torture, while also preventing the user from spamming the rack (which would make them lose a lot of blood)
About The Pull Request
Prevents basicmob torture, while also preventing the user from spamming the rack (which would make them lose a lot of blood).
Why It's Good For The Game
Spamming the rack to instantly lose all of your blood does not seem intended, and seems very harmful. Also, based on the other people you're allowed to torture, I don't think you were supposed to be able to torture simplemobs in the first place.
If you were supposed to be able to torture basic mobs, tell me, and I can fix that.
Changelog
🆑
fix: Bloodsuckers are unable to spam click the persuasion rack and lose all their blood, as part of the new clan training.
fix: Bloodsuckers will not torture basic mobs or animals, after a clan agreement involving the Animal Rights Consortium.
/:cl: